Skip to content

Update README to include releases link#16

Merged
andrewhassan merged 1 commit intomainfrom
ah.update-readme
Jun 23, 2022
Merged

Update README to include releases link#16
andrewhassan merged 1 commit intomainfrom
ah.update-readme

Conversation

@andrewhassan
Copy link
Copy Markdown
Contributor

In this PR, I've updated the README to have clearer instructions on where to find the binaries if you don't want to compile it yourself.

@andrewhassan andrewhassan force-pushed the ah.update-readme branch 3 times, most recently from d232ff1 to eb13979 Compare May 26, 2022 17:41
Comment thread README.md Outdated

This is a simple CLI (`script-runner`) which allows you to run Wasm
scripts intended for ShopifyVM. Scripts will run using
This is a simple CLI (`script-runner`) which allows you to run Shopify Functions intended for ShopifyVM. Functions will run using
Copy link
Copy Markdown

@kwringe kwringe May 27, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How about:

This simple CLI (script-runner`) lets you run Shopify Functions for ShopifyVM. It accepts a JSON input file and outputs JSON.

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not as confident about the 2nd sentence. It seems odd to me.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think maybe It accepts a JSON input file and outputs the result as JSON makes more sense. WDYT?

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Comment thread README.md Outdated
Comment thread README.md Outdated
@andrewhassan andrewhassan merged commit 8c49ae3 into main Jun 23, 2022
@andrewhassan andrewhassan deleted the ah.update-readme branch June 23, 2022 15:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants